automation: drop container name from containerize
authorDoug Goldstein <cardoe@cardoe.com>
Fri, 3 Aug 2018 14:46:47 +0000 (09:46 -0500)
committerWei Liu <wei.liu2@citrix.com>
Fri, 3 Aug 2018 15:11:22 +0000 (16:11 +0100)
This was something that existed for some scripting support for a totally
unrelated project and when I copied this script I failed to remove it so
this removes it. Build containers for Xen are best as ephemeral
environments and should just utilizes Docker's default container naming
behavior.

Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
Reviewed-by: Wei Liu <wei.liu2@citrix.com>
automation/build/README.md
automation/scripts/containerize

index 987533ac14862f6d0bae1a7d5f1478ea0f65eec8..bf113175f4b5e342b567aa6aedc058c87bac9bd3 100644 (file)
@@ -53,11 +53,6 @@ understands.
 - XEN_CONFIG_EXPERT: If this is defined in your shell it will be
   automatically passed through to the container.
 
-- CONTAINER_NAME: By default the container name is set based on the
-  container itself so that its easy to attach other terminals to your
-  container. This however prevents you from running multiple containers
-  of the same version. Override the name value to cause it to name
-  the container differently on start.
 
 Building a container
 --------------------
index f4ff24c25a2dea49e82ac1ac04c06cc040227a20..7f0eb0aff01c698ceb433f6a32d63a4093388241 100755 (executable)
@@ -65,11 +65,6 @@ if [[ -n ${SSH_AUTH_SOCK} ]]; then
     fi
 fi
 
-# if we got the CONTAINER_NAME env variable then use that for our name
-if [[ -n ${CONTAINER_NAME} ]]; then
-    name="--name ${CONTAINER_NAME}"
-fi
-
 # Figure out the base of what we want as our sources
 # by using the top of the git repo
 if [[ -z ${CONTAINER_PATH} ]]; then
@@ -85,7 +80,7 @@ exec docker run \
     -v "${HOME}/.ssh":/root/.ssh:ro \
     ${SSH_AUTH_DIR:+-v "${SSH_AUTH_DIR}":/tmp/ssh-agent} \
     ${XEN_CONFIG_EXPERT:+-e XEN_CONFIG_EXPERT=${XEN_CONFIG_EXPERT}} \
-    ${CONTAINER_ARGS} ${name} \
+    ${CONTAINER_ARGS} \
     -${termint}i --rm -- \
     ${CONTAINER} \
     ${cmd}